Skip to content

circus-evaluator: migrate evix 0.3.3 -> 2.0.1; eval fixes#133

Merged
amaanq merged 19 commits into
mainfrom
notashelf/push-tmsskmtxkqtx
Jul 17, 2026
Merged

circus-evaluator: migrate evix 0.3.3 -> 2.0.1; eval fixes#133
amaanq merged 19 commits into
mainfrom
notashelf/push-tmsskmtxkqtx

Conversation

@NotAShelf

Copy link
Copy Markdown
Member

Updates the evix to the latest version and refactors the Nix evaluation logic to use the new asynchronous streaming API provided by evix. This makes the evaluation code slightly simpler and with my latest changes in nix-bindings the error handling for evaluations is improved slightly.

Change-Id: I6d77fd62f962d4b2ad9c7f42d8de44356a6a6964

@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch from 0335dfe to 8cab644 Compare June 27, 2026 15:59
@NotAShelf
NotAShelf marked this pull request as draft July 2, 2026 20:13
@NotAShelf

Copy link
Copy Markdown
Member Author

Needs to be updated to 1.2.0

@amaanq
amaanq force-pushed the notashelf/push-tmsskmtxkqtx branch 2 times, most recently from b48b6ee to a765c28 Compare July 3, 2026 03:04
NotAShelf and others added 5 commits July 12, 2026 12:33
Replace2 blocking `evaluate_cancellable` + `spawn_blocking` with the v1
async `Session::open(config).await?` + `session.stream()` drain loop.

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I6d77fd62f962d4b2ad9c7f42d8de44356a6a6964
… count

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Ifb96aef86d352dbb2a17bf95210b99796a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: If8a502d8bfc7ef89c6e35f02bd519da46a6a6964
@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch from fd8bacb to 100ccf1 Compare July 12, 2026 09:37
@NotAShelf NotAShelf changed the title circus-evaluator: migrate evix 0.3.3 -> 1.0.2 circus-evaluator: migrate evix 0.3.3 -> 1.0.2; eval fixes Jul 12, 2026
@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch 3 times, most recently from 3b46c2f to edb8f00 Compare July 12, 2026 12:06
@NotAShelf NotAShelf changed the title circus-evaluator: migrate evix 0.3.3 -> 1.0.2; eval fixes circus-evaluator: migrate evix 0.3.3 -> 2.0.0; eval fixes Jul 12, 2026
amaanq and others added 8 commits July 12, 2026 15:30
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Icb2035c611ac45a66ec80691be4b61c36a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I1c3305276273f9ebbb74a2078c6930506a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Id8b524cf17c20559b895b64cc8529e276a6a6964
…t/cancel helpers

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I474a60ef38b56128739fa94a433b77286a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I669a11f7e17036f194dfb2e04eb1636e6a6a6964
…cellation poller

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I9e60a3414ad8ea1e671a172dbc0fc79c6a6a6964
…art via dashboard

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Ide6d8fe415cfff159577984f106af7286a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: If7d6a7875f9678089bd06e1e7a2abdc96a6a6964
@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch from edb8f00 to b3643a9 Compare July 12, 2026 12:32
@NotAShelf
NotAShelf marked this pull request as ready for review July 12, 2026 12:38
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Ife1808aa35d758f49b1a537be6075c7e6a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I9c1f5447d4865b245b0b85fd1fde3d096a6a6964
@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch from 5d8c584 to 0a86f3f Compare July 13, 2026 14:22
@NotAShelf NotAShelf changed the title circus-evaluator: migrate evix 0.3.3 -> 2.0.0; eval fixes circus-evaluator: migrate evix 0.3.3 -> 2.0.1; eval fixes Jul 13, 2026
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I722e9447acfb64dd898c540c2a164a8a6a6a6964
@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch from 0a86f3f to 91228a7 Compare July 14, 2026 04:56
@NotAShelf
NotAShelf requested a review from amaanq July 14, 2026 05:18
Comment thread crates/common/src/models.rs
Comment thread crates/evaluator/src/eval_loop.rs Outdated
Comment thread crates/evaluator/src/eval_loop.rs Outdated
Comment thread crates/common/src/repo/evaluations.rs
Comment thread crates/server/src/routes/dashboard/admin.rs
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I061874ad8c52cd82a427d598887494ef6a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I18c89d26f662d132783545f9ed13582a6a6a6964
@NotAShelf
NotAShelf force-pushed the notashelf/push-tmsskmtxkqtx branch from 8331f0d to 3189d47 Compare July 16, 2026 06:39
amaanq
amaanq previously approved these changes Jul 17, 2026
Comment thread crates/common/src/repo/evaluations.rs
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I119568674e31cc7597a040d926ac4cec6a6a6964
@amaanq
amaanq added this pull request to the merge queue Jul 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Jul 17, 2026
@amaanq
amaanq merged commit 6cebed2 into main Jul 17, 2026
31 checks passed
@amaanq
amaanq deleted the notashelf/push-tmsskmtxkqtx branch July 17, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants